[ADD, PORT] Индукторы и Мегабатареи - #10
Conversation
github-actions
Bot
added
Статус: Требуется проверка
🦈 C# 🦈
📄 Локализация 📄
🎨 Спрайты 🎨
🐤 YML 🐤
labels
Summary of ChangesHello @KashRas2, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! Этот пул-реквест значительно расширяет систему управления энергией в игре, добавляя новые инструменты и высокоемкие источники питания. Основная цель — предоставить игрокам больше стратегических возможностей в области энергоснабжения, вводя индукторы для беспроводной передачи энергии и мегабатареи для хранения больших объемов заряда. Изменения также включают новые рецепты для крафта и технологии для исследования, что обогащает игровой процесс и баланс, разделяя контент между директориями LP и ADT для лучшей организации. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
RSI Diff Bot; head commit 9499569 merging into 9d76a37 Resources/Textures/_ADT/Objects/Devices/megacells.rsi
Resources/Textures/_ADT/Structures/Power/megacharger.rsi
Resources/Textures/_LP/Objects/Tools/inducer-engi.rsi
Resources/Textures/_LP/Objects/Tools/inducer-rnd.rsi
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
Этот PR добавляет индукторы и мегабатареи, перенесенные из другой сборки. Код в целом выглядит хорошо, но есть несколько моментов, которые требуют внимания.
В InducerSystem.cs есть логика, которую можно упростить, и потенциально опасный блок else, который выполняет ручное управление зарядом батареи.
В файлах прототипов megapowercells.yml есть избыточные поля name и description с временными значениями, которые следует удалить, чтобы работала локализация. Также есть дублирование тега.
Наконец, в нескольких .ftl файлах отсутствует перевод строки в конце, что может вызвать проблемы с некоторыми инструментами.
Я оставил конкретные комментарии с предложениями по исправлению.
| else | ||
| { | ||
| _battery.SetCharge(target, targetBattery.CurrentCharge + energyToTransfer, targetBattery); | ||
| _battery.SetCharge(slot.Item.Value, sourceBattery.CurrentCharge - energyToTransfer, sourceBattery); | ||
| args.Repeat = false; | ||
| } |
There was a problem hiding this comment.
Блок else здесь выглядит подозрительно и потенциально может привести к ошибкам. Метод _battery.TryUseCharge не должен возвращать false, так как energyToTransfer уже ограничен текущим зарядом sourceBattery. Если TryUseCharge все же вернет false (например, из-за состояния гонки), то ручное изменение заряда в обход API BatterySystem является некорректным. Это может привести к рассинхронизации состояния. Рекомендуется убрать манипуляции с зарядом из этого блока.
else
{
args.Repeat = false;
}| name: megacell small | ||
| description: damnation |
There was a problem hiding this comment.
Поля name и description содержат значения-заполнители. Поскольку локализация для этих сущностей определена в файлах .ftl (например, ent-ADTMegaCellSmall), эти поля следует удалить. В противном случае в игре будут отображаться "megacell small" и "damnation" вместо локализованных строк. Это относится ко всем прототипам мегабатарей в этом файле (ADTMegaCellSmall, ADTMegaCellMedium, ADTMegaCellHigh, ADTMegaCellHyper, ADTMegaCellInfinite).
| if (targetBattery.CurrentCharge < targetBattery.MaxCharge * 0.95f) | ||
| { | ||
| args.Repeat = true; | ||
| } | ||
| else | ||
| { | ||
| args.Repeat = false; | ||
| } |
There was a problem hiding this comment.
Этот блок if/else можно упростить до одной строки для улучшения читаемости и краткости кода.
args.Repeat = targetBattery.CurrentCharge < targetBattery.MaxCharge * 0.95f;| .desc = { ent-ADTMegaCellHyper.desc } | ||
| ent-ADTMegaCellInfinite = квантовая мегабатарея | ||
| .desc = Абсолютная вершина энергетических технологий. Квантовая матрица внутри корпуса обеспечивает практически неисчерпаемый запас энергии, нарушая все известные законы термодинамики. | ||
| .suffix = Бесконечный |
There was a problem hiding this comment.
В конце файла отсутствует символ новой строки. Некоторые инструменты и системы контроля версий могут некорректно обрабатывать файлы без завершающей новой строки. Рекомендуется добавить ее.
| .suffix = Бесконечный | |
| .suffix = Бесконечный | |
| @@ -0,0 +1,2 @@ | |||
| ent-ADTMegaCellRecharger = зарядник мегабатарей | |||
| .desc = Модифицированный зарядник промышленного класса, предназначенный исключительно для зарядки мегабатарей. Оснащён контактным основанием и усиленными энергоканалами для безопасной и эффективной передачи электричества. | |||
There was a problem hiding this comment.
В конце файла отсутствует символ новой строки. Некоторые инструменты и системы контроля версий могут некорректно обрабатывать файлы без завершающей новой строки. Рекомендуется добавить ее.
| .desc = Модифицированный зарядник промышленного класса, предназначенный исключительно для зарядки мегабатарей. Оснащён контактным основанием и усиленными энергоканалами для безопасной и эффективной передачи электричества. | |
| .desc = Модифицированный зарядник промышленного класса, предназначенный исключительно для зарядки мегабатарей. Оснащён контактным основанием и усиленными энергоканалами для безопасной и эффективной передачи электричества. | |
| - type: Tag | ||
| tags: | ||
| - ADTMegaCell |
There was a problem hiding this comment.
Этот компонент Tag с тегом ADTMegaCell является избыточным. Прототип ADTMegaCellSmall наследуется от ADTBaseMegaCell, который уже содержит этот тег. Дублирование следует удалить для упрощения поддержки.
BL02DL
changed the title
|
@Evgencheg |
3 participants
Описание PR
Перенес индукторы и мегабатареи с сборки вологодских промыслов.
Почему / Баланс
Выполнил данную мини-предложку по просьбе Василисы, ничего более.
Технические детали
Уважаемый ревьювер наверное обратил внимание на мешанину в файлах. То LP, то ADT. Поэтому внесу ясность:
Так же хотел бы уточнить, что я специально не вносил крафты в автолаты в связи с этими новомодными протолатами отделов. Как только будет внесена ясность, то я внесу крафты куда надо.
Медиа
Требования
Критические изменения
Список изменений
🆑 StasNeStasNe & KashRas2